Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#173 apply job post #168

Merged
merged 1 commit into from
Dec 3, 2024
Merged

#173 apply job post #168

merged 1 commit into from
Dec 3, 2024

Conversation

jkarenzi
Copy link
Contributor

@jkarenzi jkarenzi commented Nov 29, 2024

What does this PR do?

  • This PR addresses requested feedback and implements the necessary changes to improve the application's functionality and user experience.

Description of the task to be completed

  • Removed the 'submitted' status to streamline application tracking.
  • Enabled adding comments for applications marked with the 'rejected' status, providing clarity on rejection reasons.

How can this be manually tested?

  • Clone repo
  • Checkout branch fix-applicant-process
  • Make sure that you set the environment variables referring to .env.example
  • Run npm install, then npm run dev to check if the app is running successfully
  • login as admin with username: [email protected] and password: password123
  • Applying to a job post using the createNewJobApplication mutation. The status returned should be 'under review', and not 'submitted'
  • Reject job application and add comments when rejecting an applicant, through the changeApplicationStatus mutation

- remove 'submitted' status
- add comments for 'rejected status'
@ceelogre ceelogre temporarily deployed to fix-jobpost-apply - atlp-devpulse-bn PR #168 December 2, 2024 12:53 — with Render Destroyed
@render render bot temporarily deployed to fix-jobpost-apply - dev-pulse-bn PR #168 December 2, 2024 12:53 Destroyed
@SaddockAime SaddockAime linked an issue Dec 2, 2024 that may be closed by this pull request
@SaddockAime SaddockAime changed the title Fix apply job post #173 apply job post Dec 2, 2024
@jkarenzi jkarenzi self-assigned this Dec 3, 2024
@ncutixavier ncutixavier merged commit e7f3474 into develop Dec 3, 2024
1 check passed
@ncutixavier ncutixavier deleted the fix-jobpost-apply branch December 3, 2024 13:35
bahati10 pushed a commit that referenced this pull request Dec 3, 2024
- remove 'submitted' status
- add comments for 'rejected status'
bahati10 pushed a commit that referenced this pull request Dec 6, 2024
- remove 'submitted' status
- add comments for 'rejected status'
Salim-54 pushed a commit that referenced this pull request Dec 9, 2024
* Application cycle process on applicant

* implement Technical Interview invitation

* implement Technical Interview invitation

* Implement Interview inviation

* Pipeline envs (#163)

* fix: remove the rm step

* test: run on PR

* test: test pusher cluster env

* fix: add notification envs

* test: add pusher key

* fix: add the pusher app key

* fix: log container start output

* fix: remove the e flag

* fix: revert changes

* fix: re-add the pusher app key

* fix: add pusher app key value

* fix: typo

* fix: test env file

* fix: syntax

* fix: remove the env from run cmd

* fix: revert changes

* fix: revert changes

* fix: wrap run cmd in an if statement

* Revert "test: test pusher cluster env"

This reverts commit 57b2bbe.

* test: create env file

* test: add error handling

* fix: run on push (#176)

* fix(apply-jobpost): incorporate feedback (#168)

- remove 'submitted' status
- add comments for 'rejected status'

* fix: don't exit on cmd failure (#177)

* fix: don't exit on cmd failure

* fix: re-add values

* fix: add new variables

* test: add container name

* fix: match ports

* fix: add devpulse email

* fix: port

* fix: port

* fix: remove container name

* fix: remove extraneous node env

* fix: add jwt key

* fix: run on push

* #170 An author will be able to update a blog (#171)

* fix (170): an author will be able to update a blog

* fix (170): allow all users to create and update blogs

* Fix to change status in application stage process (#165)

* Ft: Enhance Backend documentation (#164)

* fix: improve on notifications (#175)

* Application cycle process on applicant

* Application cycle process on applicant

* Fixing conflicts and removing unnecessary files

---------

Co-authored-by: Aime-Patrick <[email protected]>
Co-authored-by: Samuel Nishimwe <[email protected]>
Co-authored-by: Joslyn Manzi Karenzi <[email protected]>
Co-authored-by: mutsinziisaac <[email protected]>
Co-authored-by: Emmanuel MUGISHA <[email protected]>
Co-authored-by: ISHIMWE Jean Baptiste <[email protected]>
Co-authored-by: Christian Iradukunda <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

apply job post
8 participants